Skip to content

ci: skip GitHub API-heavy jobs on non-main branches#24370

Merged
pelikhan merged 1 commit intomainfrom
copilot/disable-ci-jobs-non-main-branches
Apr 3, 2026
Merged

ci: skip GitHub API-heavy jobs on non-main branches#24370
pelikhan merged 1 commit intomainfrom
copilot/disable-ci-jobs-non-main-branches

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 3, 2026

PR branch CI runs were hitting GitHub API rate limits due to several jobs making external API calls on every push. These jobs are only meaningful on main (release validation, DIFC proxy smoke tests, live API tests).

Changes

Added if: github.ref == 'refs/heads/main' to four jobs:

  • integration-release-availability — calls gh release view against 3 external repos
  • js-integration-live-api — live GitHub API test (frontmatter_hash_github_api.test.cjs)
  • sh-difc-proxy — pulls from GHCR and calls GitHub API through the DIFC proxy
  • sh-gh-host-pr-checkout-repro — DIFC-style proxy repro test

Follows the same pattern already used by bench, actions-build, and security-scan.

@pelikhan
Copy link
Copy Markdown
Collaborator

pelikhan commented Apr 3, 2026

create a pr

@pelikhan pelikhan marked this pull request as ready for review April 3, 2026 20:14
Copilot AI review requested due to automatic review settings April 3, 2026 20:14
@pelikhan pelikhan merged commit bf6e774 into main Apr 3, 2026
45 of 46 checks passed
@pelikhan pelikhan deleted the copilot/disable-ci-jobs-non-main-branches branch April 3, 2026 20:14
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates CI to avoid GitHub API rate-limit issues by skipping API-heavy jobs unless the workflow is running on the main branch.

Changes:

  • Added a main-only job condition (if: github.ref == 'refs/heads/main') to js-integration-live-api.
  • Added a main-only job condition to integration-release-availability.
  • Added a main-only job condition to sh-difc-proxy and sh-gh-host-pr-checkout-repro.
Show a summary per file
File Description
.github/workflows/ci.yml Gates four GitHub API-heavy jobs to only run on refs/heads/main, matching the existing pattern used by other main-only jobs.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants